Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Changed links and naming to valkey instead of redis #389

Merged
merged 9 commits into from
May 2, 2024

Conversation

simkusr
Copy link
Contributor

@simkusr simkusr commented Apr 26, 2024

This is a minor change where only naming and links now points properly to valkey.

Fixes #388

Signed-off-by: Rolandas Šimkus <rolandas@simkus.io>
@simkusr simkusr marked this pull request as draft April 26, 2024 11:11
Signed-off-by: Rolandas Šimkus <rolandas@simkus.io>
Signed-off-by: Rolandas Šimkus <rolandas@simkus.io>
same in documentation

Signed-off-by: Rolandas Šimkus <rolandas@simkus.io>
@simkusr
Copy link
Contributor Author

simkusr commented Apr 26, 2024

non trivial naming conventions were changed from redis to valkey.

@simkusr simkusr marked this pull request as ready for review April 26, 2024 11:33
@simkusr
Copy link
Contributor Author

simkusr commented Apr 26, 2024

There are still left redis naming, but those ones require additional testing for scripts and more, it's different then this simple PR. With this PR I had an idea to rename from Redis to Valkey where the most we can see it, other require code changes so its out of this PR scope.

00-RELEASENOTES Outdated Show resolved Hide resolved
COPYING Outdated Show resolved Hide resolved
MANIFESTO Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
deps/hiredis/COPYING Outdated Show resolved Hide resolved
Signed-off-by: simkusr <rolandas.s@wilibox.com>
@simkusr
Copy link
Contributor Author

simkusr commented Apr 29, 2024

@madolson thanks for review, did the rollback and changes you mentioned, thanks again!

deps/README.md Outdated Show resolved Hide resolved
COPYING Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
deps/README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
simkusr and others added 4 commits May 2, 2024 12:42
Signed-off-by: simkusr <rolandas@simkus.io>
Signed-off-by: simkusr <rolandas@simkus.io>
Signed-off-by: Viktor Söderqvist <viktor.soderqvist@est.tech>
Copy link

codecov bot commented May 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.43%. Comparing base (b0d5a0f) to head (e606834).
Report is 10 commits behind head on unstable.

Additional details and impacted files
@@             Coverage Diff              @@
##           unstable     #389      +/-   ##
============================================
+ Coverage     68.33%   68.43%   +0.09%     
============================================
  Files           108      109       +1     
  Lines         61563    61671     +108     
============================================
+ Hits          42072    42206     +134     
+ Misses        19491    19465      -26     

see 20 files with indirect coverage changes

@zuiderkwast zuiderkwast added the rebranding Valkey is not Redis label May 2, 2024
@zuiderkwast zuiderkwast merged commit 68ca258 into valkey-io:unstable May 2, 2024
17 checks passed
PingXie pushed a commit to PingXie/valkey that referenced this pull request May 2, 2024
This is a minor change where only naming and links now points properly
to valkey.

Fixes valkey-io#388

---------

Signed-off-by: Rolandas Šimkus <rolandas@simkus.io>
Signed-off-by: simkusr <rolandas.s@wilibox.com>
Signed-off-by: simkusr <rolandas@simkus.io>
Signed-off-by: Viktor Söderqvist <viktor.soderqvist@est.tech>
Co-authored-by: simkusr <rolandas.s@wilibox.com>
Co-authored-by: Viktor Söderqvist <viktor.soderqvist@est.tech>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rebranding Valkey is not Redis
Projects
None yet
Development

Successfully merging this pull request may close these issues.

naming conventions and links should be changed from redis to valkey
3 participants